Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix flash target in contrib/Makefile #262

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

agren
Copy link
Contributor

@agren agren commented Sep 6, 2024

Description

Fixes an issue with running make -C contrib flash (Command found in the dev guide). The command exits with the following error:
Error: workdir "/build/hw/application_fpga" does not exist on container

Fixed by setting the working directory to /build.

Previously the working directory was set to /build/hw/application_fpga. But that path does not exist since hw/application_fpga is already mounted to /build. Thus resulting in an error.

Type of change

  • Bugfix (non breaking change which resolve an issue)

Submission checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my changes
  • I have tested and verified my changes on target
  • My changes are well written and CI is passing
  • I have squashed my work to relevant commits and rebased on main for linear history

Setting the working directory to a directory that exist.

Previously the working directory was set to /build/hw/application_fpga.
But that path does not exist since hw/application_fpga was mounted to
/build. Thus resulting in an error:
'Error: workdir "/build/hw/application_fpga" does not exist on
container'
@agren agren self-assigned this Sep 6, 2024
@dehanj dehanj requested a review from jthornblad October 9, 2024 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants